Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0] [ADD] egd_stock_picking_custom: add new module #2

Closed
wants to merge 3 commits into from

Conversation

rodrigonevest
Copy link
Contributor

@rodrigonevest rodrigonevest commented Aug 29, 2023

@marcelsavegnago feito as alterações.

Acredito que o erro era por conta da dependência do módulo hr_personal_equipment_request, onde foi implementado.

A descrição foi implementada também.

@rodrigonevest rodrigonevest marked this pull request as draft August 29, 2023 21:21
@codecov
Copy link

codecov bot commented Aug 29, 2023

Codecov Report

Merging #2 (b826073) into 14.0 (3d5f2c8) will increase coverage by 3.98%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             14.0       #2      +/-   ##
==========================================
+ Coverage   38.36%   42.35%   +3.98%     
==========================================
  Files           7       11       +4     
  Lines         159      170      +11     
  Branches       35       37       +2     
==========================================
+ Hits           61       72      +11     
  Misses         98       98              
Files Changed Coverage Δ
egd_stock_picking_custom/__init__.py 100.00% <100.00%> (ø)
egd_stock_picking_custom/models/__init__.py 100.00% <100.00%> (ø)
..._picking_custom/models/egd_stock_picking_custom.py 100.00% <100.00%> (ø)
...king_custom/models/egd_stock_product_lot_custom.py 100.00% <100.00%> (ø)

@rodrigonevest rodrigonevest force-pushed the 14.0-add-egd_stock_picking_custom branch 3 times, most recently from 6fa2b67 to 3ac3e5a Compare September 4, 2023 22:18
@rodrigonevest rodrigonevest force-pushed the 14.0-add-egd_stock_picking_custom branch 2 times, most recently from cc3b0b6 to af2a18b Compare September 7, 2023 11:06
@rodrigonevest rodrigonevest marked this pull request as ready for review September 7, 2023 11:09
@rodrigonevest rodrigonevest force-pushed the 14.0-add-egd_stock_picking_custom branch 3 times, most recently from cfc07b0 to 216e1b8 Compare September 9, 2023 03:33
@marcelsavegnago
Copy link
Contributor

@rodrigonevest faz um rebase por favor

@marcelsavegnago
Copy link
Contributor

Favor incluir pasta README e arquivos USAGE.rst, DESCRIPTIOn.rst e o de contribuintes

Favor inserir pasta static tbm

@kaynnan favor acompanhar e apoiar o @rodrigonevest

@marcelsavegnago
Copy link
Contributor

Obs.. Inclua também no descritivo do módulo o que está implementando. Em ingles.

@marcelsavegnago
Copy link
Contributor

image

acredito que falta uma dependencia ai.. quem implementa o campo equipment_request_id ?

@rodrigonevest rodrigonevest force-pushed the 14.0-add-egd_stock_picking_custom branch 3 times, most recently from ff3261f to 4cdd150 Compare September 19, 2023 00:35
@marcelsavegnago
Copy link
Contributor

marcelsavegnago commented Sep 19, 2023

Acho que ainda falta dependencia. Você criou uma base nova e instalando apenas este módulo ? se assim rodar bonitinho é porque as dependencias estão de certa forma configuradas corretamente.

image

@kaynnan vc chegou testar isso localmente ?

@marcelsavegnago
Copy link
Contributor

Corrigi as dependencias nesta commit. @rodrigonevest dá uma olhada e caso tenha alguma dúvida por favor informe.

b826073

@kaynnan
Copy link
Contributor

kaynnan commented Sep 19, 2023

Acho que ainda falta dependencia. Você criou uma base nova e instalando apenas este módulo ? se assim rodar bonitinho é porque as dependencias estão de certa forma configuradas corretamente.

image

@kaynnan vc chegou testar isso localmente ?

Ainda nâo, até as 14h dou um retorno sobre e faço o review tbm

@rodrigonevest
Copy link
Contributor Author

@marcelsavegnago feito as alterações.

Acredito que o erro era por conta da dependência do módulo hr_personal_equipment_request, onde foi implementado.

A descrição foi implementada também.

Acho que ainda falta dependência. Você criou uma base nova e instalou apenas este módulo? se assim rodar bonitinho é porque as dependências estão de certa forma definidas corretamente.

imagem

@kaynnanvocê chegou testar isso localmente ?

Esse campo deve está configurado conforme a imagem para aparecer a data de expiração:
image

@marcelsavegnago
Copy link
Contributor

@kaynnan kaynnan deleted the 14.0-add-egd_stock_picking_custom branch December 6, 2023 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants